drivers: ethernet: lan9250: add support for a random mac address #99127
Conversation
zephyrbot
requested review from
ClaCodes,
decsny,
lmajewski,
maass-hamburg and
pdgendt
dsseng
left a comment
dsseng
left a comment
There was a problem hiding this comment.
Thank you! Please find my suggestions below
ckhardin
force-pushed
the
ckhardin/add-random-mac-to-lan9250
branch
from
659344d to
c5e45ea
Compare
|
interesting - not sure how the msg wait all stack issues relate to this change - so, need to see if that failure is upstream or in this PR alone? |
Looks like a flaky test, unlikely to be caused by this PR. Someone with the CI access rights will restart the failed twister run and let you know if it is somehow relevant |
ckhardin
force-pushed
the
ckhardin/add-random-mac-to-lan9250
branch
from
c5e45ea to
fdaa24c
Compare
drivers/ethernet/eth_lan9250.c
Outdated
|
|
||
| gpio_pin_interrupt_configure_dt(&config->interrupt, GPIO_INT_EDGE_TO_ACTIVE); | ||
|
|
||
| if (config->reset.port) { |
There was a problem hiding this comment.
| if (config->reset.port) { | |
| if (config->reset.port != NULL) { |
the condition has to be a boolean. See coding guidelines
rule 85
The controlling expression of an if statement and the controlling expression of an iteration-statement shall have essentially Boolean type
There was a problem hiding this comment.
Not all the ethernet drivers are updated to the coding rules - this is a cut and paste from eth_w5500.c, so updated it for this patch.
ckhardin
force-pushed
the
ckhardin/add-random-mac-to-lan9250
branch
2 times, most recently
from
8e34711 to
1c940f7
Compare
Extend the lan9250 driver to support using a local administered unicast random mac address during init. This follows the device tree settings for zephyr_random_mac_address from other ethernet drivers for the added support. Signed-off-by: Charles Hardin <ckhardin@gmail.com>
The reset gpio field was in the config structure but was not coded into the initialization path. So, add the appropriate code to handle the gpio setup when it is defined in the device tree. Signed-off-by: Charles Hardin <ckhardin@gmail.com>
ckhardin
force-pushed
the
ckhardin/add-random-mac-to-lan9250
branch
from
1c940f7 to
93dac9c
Compare
|
4 participants
Extend the lan9250 driver to support using a local administered unicast random mac address during init. This follows the device tree settings for zephyr_random_mac_address from other ethernet drivers for the added support.